Skip to content

Fix serena-mcp-server test.sh entrypoint override for runtime verification #341

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
lpcox merged 2 commits into from
Jan 19, 2026
Merged

Fix serena-mcp-server test.sh entrypoint override for runtime verification #341

lpcox merged 2 commits into from
Jan 19, 2026

Conversation

Copy link
Contributor

Copilot AI commented Jan 19, 2026 •
edited
Loading

The Serena MCP server test suite was failing to verify language runtimes (Python, Java, Node.js, Go) because the container's ENTRYPOINT caused commands to be interpreted as arguments to serena-mcp-server rather than executed directly.

Changes

  • Test 4 (Language Runtimes): Added --entrypoint flag to override container entrypoint for direct command execution

    • Python: --entrypoint python3
    • Java: --entrypoint java
    • Node.js: --entrypoint node
    • Go: --entrypoint go

  • Test 5 (Language Servers): Updated shell command checks to use --entrypoint sh for proper execution context

Before/After

# Before - fails with "Got unexpected extra argument (go)"
docker run --rm "$CONTAINER_IMAGE" go version

# After - executes go command directly
docker run --rm --entrypoint go "$CONTAINER_IMAGE" version

All language runtime and language server verification tests now execute correctly.

Original prompt

This section details on the original issue you should resolve

<issue_title>Smoke Copilot - Issue Group</issue_title>
<issue_description># Smoke Copilot

Parent issue for grouping related issues from Smoke Copilot.

Sub-issues are automatically linked below (max 64 per parent).
</issue_description>

<agent_instructions>Please try to understand why this test is failing

✅ bash
Test go command availability

$ echo "Testing go command in serena context" && which go && go version

Testing go command in serena context
<exited with exit code 1></agent_instructions>

Comments on the Issue (you are @copilot in this section)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@lpcox
Fix serena-mcp-server test.sh to properly verify language runtimes
65a2dee 
The test script was failing because the container has an ENTRYPOINT set to
'serena-mcp-server', causing commands like 'go version' to be interpreted
as arguments to serena-mcp-server rather than executing the go command.

Fixed by adding --entrypoint flag to all language runtime verification tests:
- Python: Use --entrypoint python3
- Java: Use --entrypoint java
- Node.js: Use --entrypoint node
- Go: Use --entrypoint go
- Language servers: Use --entrypoint sh for shell commands

All tests now pass successfully.

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Copilot AI changed the title [WIP] Investigate reason for test failure in Smoke Copilot Fix serena-mcp-server test.sh entrypoint override for runtime verification Jan 19, 2026
Copilot AI requested a review from lpcox January 19, 2026 04:32
@lpcox lpcox marked this pull request as ready for review January 19, 2026 04:35
@lpcox lpcox merged commit ec7d513 into main Jan 19, 2026
1 check passed
@lpcox lpcox deleted the copilot/investigate-test-failure branch January 19, 2026 04:35
@github-actions github-actions bot mentioned this pull request Jan 19, 2026
Closed
Copilot AI mentioned this pull request Jan 19, 2026
Merged
@github-actions github-actions bot mentioned this pull request Jan 20, 2026
Closed
Copilot AI mentioned this pull request Jan 20, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lpcox lpcox Awaiting requested review from lpcox

Assignees

@lpcox lpcox

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Smoke Copilot - Issue Group

2 participants

@lpcox